Skip to content

Add get_headers stub#5140

Merged
VincentLanglet merged 4 commits intophpstan:2.1.xfrom
VincentLanglet:getHeaders
Mar 8, 2026
Merged

Add get_headers stub#5140
VincentLanglet merged 4 commits intophpstan:2.1.xfrom
VincentLanglet:getHeaders

Conversation

@VincentLanglet
Copy link
Copy Markdown
Contributor

@VincentLanglet VincentLanglet marked this pull request as ready for review March 7, 2026 11:40
@phpstan-bot
Copy link
Copy Markdown
Collaborator

This pull request has been marked as ready for review.

@VincentLanglet VincentLanglet requested a review from staabm March 7, 2026 11:40
Copy link
Copy Markdown
Contributor

@staabm staabm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

since issue-bot does not detect a fixed issue, might be worth to add a NSRT test

Copy link
Copy Markdown
Contributor

@staabm staabm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thank you. issue bot now detects a fixed issue, after you added the test.. interessting :-)

@staabm
Copy link
Copy Markdown
Contributor

staabm commented Mar 8, 2026

issue bot now detects a fixed issue, after you added the test

just see that on the previous commit - before you added a test - one of the issue-bot jobs failled, and therefore the summary was not generated.

https://github.com/phpstan/phpstan-src/actions/runs/22798301674/job/66136118584?pr=5140

so issue-bot is fine

@VincentLanglet VincentLanglet merged commit 8f3930e into phpstan:2.1.x Mar 8, 2026
639 of 648 checks passed
uekann pushed a commit to uekann/phpstan-src that referenced this pull request Mar 9, 2026
phpstan-bot pushed a commit to phpstan-bot/phpstan-src that referenced this pull request Apr 7, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants